Skip to content

Conversation

@tmacro
Copy link
Contributor

@tmacro tmacro commented Oct 27, 2025

No description provided.

@codecov
Copy link

codecov bot commented Oct 27, 2025

Codecov Report

❌ Patch coverage is 88.88889% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.61%. Comparing base (fcc2688) to head (ec4833b).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/Config.js 0.00% 1 Missing ⚠️
lib/api/apiUtils/rateLimit/client.js 94.11% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/Config.js 77.92% <0.00%> (-0.09%) ⬇️
lib/api/apiUtils/rateLimit/client.js 94.11% <94.11%> (ø)

... and 1 file with indirect coverage changes

@@                               Coverage Diff                               @@
##           improvement/CLDSRV-766/bucket_rate_limiting    #5989      +/-   ##
===============================================================================
+ Coverage                                        82.53%   82.61%   +0.07%     
===============================================================================
  Files                                              195      196       +1     
  Lines                                            12352    12370      +18     
===============================================================================
+ Hits                                             10195    10219      +24     
+ Misses                                            2157     2151       -6     
Flag Coverage Δ
ceph-backend-test 64.93% <0.00%> (-0.10%) ⬇️
file-ft-tests 65.93% <0.00%> (+4.50%) ⬆️
kmip-ft-tests 26.66% <0.00%> (-0.04%) ⬇️
mongo-v0-ft-tests 67.24% <0.00%> (-0.10%) ⬇️
mongo-v1-ft-tests 67.23% <0.00%> (-0.10%) ⬇️
multiple-backend 35.23% <0.00%> (-0.06%) ⬇️
quota-tests 31.85% <0.00%> (-0.86%) ⬇️
quota-tests-inflights 33.75% <0.00%> (-0.09%) ⬇️
unit 67.01% <88.88%> (+0.03%) ⬆️
utapi-v2-tests 32.94% <0.00%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tmacro tmacro force-pushed the improvement/CLDSRV-767/inmem_counter_config_cache branch from 84fa622 to eb4a19a Compare October 28, 2025 16:05
Base automatically changed from improvement/CLDSRV-767/inmem_counter_config_cache to improvement/CLDSRV-766/bucket_rate_limiting October 28, 2025 16:57
@tmacro tmacro force-pushed the improvement/CLDSRV-771/rate_limit_redis_client_wrapper branch from d97a15f to 3317748 Compare October 30, 2025 15:12
@tmacro tmacro force-pushed the improvement/CLDSRV-771/rate_limit_redis_client_wrapper branch 2 times, most recently from 74bd8d2 to ec4833b Compare November 3, 2025 17:23
@tmacro tmacro force-pushed the improvement/CLDSRV-771/rate_limit_redis_client_wrapper branch from ec4833b to 02c281b Compare November 5, 2025 00:51
@tmacro tmacro merged commit 6f489d3 into improvement/CLDSRV-766/bucket_rate_limiting Nov 5, 2025
5 checks passed
@tmacro tmacro deleted the improvement/CLDSRV-771/rate_limit_redis_client_wrapper branch November 5, 2025 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants